src/utils/preflightChecks.tsx (1)

19-21: Centralize the skip-flag check (and optionally emit a “preflight skipped” event).

The same env key and condition are duplicated in checkEndpoints() and _temp(). Extract a shared constant/helper to avoid drift, and consider logging a dedicated skip event for observability.

♻️ Suggested refactor

+const SKIP_PREFLIGHT_ENV = 'CLAUDE_CODE_SKIP_PREFLIGHT_CHECK';
+const shouldSkipPreflight = () => isEnvTruthy(process.env[SKIP_PREFLIGHT_ENV]);

 async function checkEndpoints(): Promise<PreflightCheckResult> {
-  if (isEnvTruthy(process.env.CLAUDE_CODE_SKIP_PREFLIGHT_CHECK)) { //Bypasses preflight checks if the environment variable is set.
-    return { success: true }; //return true
+  if (shouldSkipPreflight()) {
+    // Optional: logEvent('tengu_preflight_check_skipped', { env: SKIP_PREFLIGHT_ENV });
+    return { success: true };
   }

 function _temp() {
-  if (isEnvTruthy(process.env.CLAUDE_CODE_SKIP_PREFLIGHT_CHECK)) {
-    return;// Skip preflight checks if the corresponding environment variable is enabled.
+  if (shouldSkipPreflight()) {
+    return;
   }
   return process.exit(1);
 }

Also applies to: 153-155

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/utils/preflightChecks.tsx` around lines 19 - 21, Extract the duplicated
environment check into a single exported helper (e.g., const SKIP_PREFLIGHT_ENV
= 'CLAUDE_CODE_SKIP_PREFLIGHT_CHECK' and function shouldSkipPreflight() that
calls isEnvTruthy(process.env[SKIP_PREFLIGHT_ENV])) and replace the inline
checks in the current file and in checkEndpoints() and _temp() to call
shouldSkipPreflight(); also emit or log a single "preflight skipped"
observability event from that helper (or have it call a provided logger/emitter)
so all callers get a consistent skip message.